Skip to content

v2.4.0: fix critical bugs, optimize cache, clean dead code#9

Merged
lordixir merged 1 commit intosercanarga:mainfrom
lordixir:main
Feb 22, 2026
Merged

v2.4.0: fix critical bugs, optimize cache, clean dead code#9
lordixir merged 1 commit intosercanarga:mainfrom
lordixir:main

Conversation

@lordixir
Copy link
Collaborator

Bug Fixes:

  • Embed VERSION with go:embed (fixes path dependency)
  • Fix race condition in ResolveSite with sync.Once
  • Fix DNS hostname key mismatch (protocol prefix stripping)
  • Use flag.Visit for correct CLI flag override logic
  • Use filepath.Join for cross-platform paths
  • Fix flaky DNS test (network-dependent)

Improvements:

  • Optimize cache.IsScanned O(n) to O(1) with map
  • Rename FoundedWebsites to FoundWebsites with backward compat
  • Add domain validation before processing
  • Update Chrome 131 comment to Chrome 135

Removed:

  • Unused GetTransport, AddSmartJitter, SaveConfigFile, ParseDNSServers
  • Unused BatchReverseDNSWithResults, DNSResult, PoolMetrics

Docs:

  • Update CHANGELOG, README, CONTRIBUTING for v2.4.0
  • Add missing CLI flags to README
  • Update project structure in CONTRIBUTING

Bug Fixes:
- Embed VERSION with go:embed (fixes path dependency)
- Fix race condition in ResolveSite with sync.Once
- Fix DNS hostname key mismatch (protocol prefix stripping)
- Use flag.Visit for correct CLI flag override logic
- Use filepath.Join for cross-platform paths
- Fix flaky DNS test (network-dependent)

Improvements:
- Optimize cache.IsScanned O(n) to O(1) with map
- Rename FoundedWebsites to FoundWebsites with backward compat
- Add domain validation before processing
- Update Chrome 131 comment to Chrome 135

Removed:
- Unused GetTransport, AddSmartJitter, SaveConfigFile, ParseDNSServers
- Unused BatchReverseDNSWithResults, DNSResult, PoolMetrics

Docs:
- Update CHANGELOG, README, CONTRIBUTING for v2.4.0
- Add missing CLI flags to README
- Update project structure in CONTRIBUTING
@lordixir lordixir merged commit 2376097 into sercanarga:main Feb 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant